Greptile Summary

This PR renames the cloud_only pytest marker to access_tokens, reflecting a more precise semantic: the marker now specifically identifies access-token-related tests rather than broadly "cloud-only" tests. As a side effect, test_get_basin_config loses its marker entirely and will now run in the s2-lite integration test suite, which appears intentional since that test doesn't involve access tokens and is presumably supported on s2-lite.

Confidence Score: 5/5

Safe to merge — pure marker rename with intentional, low-risk behavioural change for one test.

All findings are P2 or lower. The rename is consistent across pytest.ini, the CI workflow, and all test annotations. The removal of the marker from test_get_basin_config appears deliberate and correct.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[pytest test run] --> B{Marker filter}
    B --> C["local-checks\nnot (account or basin or stream or metrics)"]
    B --> D["s2-cloud integration\naccount or basin or stream or metrics"]
    B --> E["s2-lite integration\n(account or basin or stream) and not access_tokens"]

    C --> F[Unit tests only]
    D --> G[All account/basin/stream/metrics tests\nincl. access_tokens tests]
    E --> H[Account/basin/stream tests\nexcl. access_tokens tests]

    style G fill:#d4edda
    style H fill:#fff3cd

_{Reviews (1): Last reviewed commit: "initial commit" | Re-trigger Greptile}